This repository was archived by the owner on Sep 15, 2025. It is now read-only.
forked from microsoft/vscode
-
Notifications
You must be signed in to change notification settings - Fork 0
update #1
Open
denis0001-dev
wants to merge
2,644
commits into
Toolbox-io:main
Choose a base branch
from
microsoft:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
update #1
+115,667
−62,950
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Switches from `node.parent.replaceChild(x, node)` to `node.replaceWith(x)`. This lets us avoid having to check that the parent exists
Update not_team_members
feat: add 'hiddenAfterComplete' option to tool's presentation prop
* fix for 267145 * fix for 267145: correcting variable name * Update src/vs/workbench/contrib/debug/browser/repl.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Connor Peet <copeet@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix any cast * Fix some 'as any'
Use `replaceWith` in to avoid not null assertions
Remove `any` casts in some built-in extensions
Fix checking $traceid for empty args
Remove a few more `any` casts
Fix `any` cast in code actions
… args Part of #269213 This prevents accessing arbitrary properties on the args without first doing a type check or type assertion
…y` for args Part of #269213 Updates the command service to use `unknown` for args. Does not yet update `ICommandHandler` as this will require many more changes
Update caller side of CommandService to use `unknown` instead of `any` for args
Fix some `any` casts and `any` usage in instantiation service
… Coverage (#269505) * Add navigation for uncovered lines in test coverage Implements [#258967](https://github.com/obrobrio2000/vscode/issues/258967) - Add commands to navigate to next/previous uncovered lines - Add navigation buttons to coverage toolbar widget - Add keyboard shortcuts (Alt+F9 / Shift+Alt+F9) - Navigation wraps around at file boundaries - Buttons appear in coverage toolbar and editor title menu This feature helps developers efficiently review coverage gaps by allowing them to jump between uncovered lines without manual scrolling through the file. Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> * Update src/vs/workbench/contrib/testing/browser/codeCoverageDecorations.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix navigation logic and add sorting for uncovered lines - Add proper sorting of missed lines by line number - Fix corrupted control flow in previous/back navigation - Add missing variable declarations - Remove duplicate/incorrect conditional blocks This fixes the navigation to work correctly with properly ordered missed lines. Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> * feat: add navigation commands for uncovered lines in code coverage Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> * fix: add missing ID property and remove unused method - Add static ID property to CodeCoverageDecorations class - Remove unused isMissedLine method (logic handled inline) - Fixes TypeScript compilation errors Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> * fix: update CodeCoverageDecorations ID to use Testing constant Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> * fix: refactor navigation titles for uncovered lines to use constants Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> --------- Signed-off-by: Giovanni Magliocchetti <giovimag123@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Tries to use the standard markdown actionHandler in more cases As part of this, switches some callers from the low level `renderMarkdown` function to the `IMarkdownRendererService`
Avoid using custom markdown actionHandlers in a few more cases
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…s picker (#270450) * Add a button to view extension details from Managed Trusted Extensions picker * PR feedback
We want to prevent mistaken changes that do something like this: ```ts foo.onEvent = () => { ... }; ``` When they almost always mean: ```ts foo.onEvent(() => { ... }) ```
Co-authored-by: Dmitriy Vasyura <dmitriv@microsoft.com>
Mark most `Event` properties as `readonly`
* Add prompt support to QuickPick * Update src/vs/platform/quickinput/browser/quickInput.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Fix incorrect suggestion * PR feedback * Merge branch 'main' into dev/dmitriv/fix-78335 * PR feedback * Merge branch 'main' into dev/dmitriv/fix-78335 * Fix build break.
…tion Fix debug exception navigation appearing in all editor groups
…270476) * Changes to remove the disabled extensionMCP server from cached servers. * Adding tests for extensionMCPDiscovery * removing extra lines * removing extra lines * adding more tests
…-JSON responses (#270579) * Initial plan * Add Content-Type check before JSON parsing in _getAuthorizationServerMetadata Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> * Improve comment explaining Content-Type check fallback behavior Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> * Fix: Check both status AND content-type for all metadata URL attempts Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> * Refactor: Try parsing JSON payload instead of checking Content-Type Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> * Refactor: Use early returns instead of nested if blocks Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> * add a log --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TylerLeonhardt <2644648+TylerLeonhardt@users.noreply.github.com> Co-authored-by: Tyler Leonhardt <tyleonha@microsoft.com>
* Retry Auth on a 401 If we have an Authorization header and still get a 401 when trying to connect to an MCP Server, we should retry with a new auth registration. This fix was for the local MCP dev loop, where you may use an in-memory OAuth server. If you kill the server and start it again, any tokens will be rejected. This PR allows us to react to that by re-doing the auth flow. * Update src/vs/workbench/api/common/extHostMcp.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Update chat code block languages when languages change
Fix chat session view menu context key.
* Changes to remove the disabled extensionMCP server from cached servers. * Adding tests for extensionMCPDiscovery * removing extra lines * removing extra lines * adding more tests * Fixing the tests for build error
* Changes to remove the disabled extensionMCP server from cached servers. * Adding tests for extensionMCPDiscovery * removing extra lines * removing extra lines * adding more tests * Fixing the tests for build error * Fixing the tests for build error
Fix auto approve links for new rules
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.